Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating signing tasks to use .NET 6 #25021

Merged
merged 4 commits into from
Nov 23, 2023
Merged

Conversation

Benjin
Copy link
Contributor

@Benjin Benjin commented Nov 22, 2023

Successful run (minus release)

@coveralls
Copy link

coveralls commented Nov 22, 2023

Pull Request Test Coverage Report for Build 6961800062

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.001%) to 41.569%

Files with Coverage Reduction New Missed Lines %
extensions/notebook/src/book/bookTreeView.ts 1 36.91%
Totals Coverage Status
Change from base Build 6951614847: -0.001%
Covered Lines: 30791
Relevant Lines: 69372

💛 - Coveralls

@kisantia
Copy link
Contributor

Should the STS pipeline also be updated to use this updated signing task? Doesn't look like the STS pipelines are failing yet, but might be good to update those to use the new task too

@Benjin
Copy link
Contributor Author

Benjin commented Nov 22, 2023

@kisantia

Should the STS pipeline also be updated to use this updated signing task? Doesn't look like the STS pipelines are failing yet, but might be good to update those to use the new task too

Yup, can do.

@cheenamalhotra
Copy link
Member

I'll proceed to merge, since Product builds have been failing..

@cheenamalhotra cheenamalhotra merged commit d3de6ea into main Nov 23, 2023
19 checks passed
@cheenamalhotra cheenamalhotra deleted the benjin/signingUpdate branch November 23, 2023 18:36
cheenamalhotra pushed a commit that referenced this pull request Jan 2, 2024
* Updating signing tasks to EsrpCodeSigning@3

* Updating .NET versions

* removing UseDotNet steps

* Restoring UseDotNet for linux builds
kisantia pushed a commit that referenced this pull request Jan 3, 2024
* Updating signing tasks to EsrpCodeSigning@3

* Updating .NET versions

* removing UseDotNet steps

* Restoring UseDotNet for linux builds

Co-authored-by: Benjin Dubishar <[email protected]>
kisantia pushed a commit that referenced this pull request Jan 4, 2024
* Updating signing tasks to EsrpCodeSigning@3

* Updating .NET versions

* removing UseDotNet steps

* Restoring UseDotNet for linux builds

Co-authored-by: Benjin Dubishar <[email protected]>
kisantia pushed a commit that referenced this pull request Jan 8, 2024
* Updating signing tasks to EsrpCodeSigning@3

* Updating .NET versions

* removing UseDotNet steps

* Restoring UseDotNet for linux builds

Co-authored-by: Benjin Dubishar <[email protected]>
kisantia added a commit that referenced this pull request Jan 8, 2024
* Codesign unis000.exe with ESRP client (#25089)

* update version to 1.41.3

* Use python 3.11.x in GitHub workflows (#25019) (#25113)

* Use python v3.11 for Darwin (#24976) (#25115)

Co-authored-by: Lewis Sanchez <[email protected]>

* Updating signing tasks to use .NET 6 (#25021) (#25160)

* Updating signing tasks to EsrpCodeSigning@3

* Updating .NET versions

* removing UseDotNet steps

* Restoring UseDotNet for linux builds

Co-authored-by: Benjin Dubishar <[email protected]>

* Use python 3.11 (#24646)

* Fix duplicate setup-python in windows ci

* change node version back to 16 for github workflows

---------

Co-authored-by: Cheena Malhotra <[email protected]>
Co-authored-by: Lewis Sanchez <[email protected]>
Co-authored-by: Benjin Dubishar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants